Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Email Notifications [mimics main branch] #140

Closed
wants to merge 106 commits into from
Closed

Email Notifications [mimics main branch] #140

wants to merge 106 commits into from

Conversation

coilysiren
Copy link
Contributor

@coilysiren coilysiren commented Oct 4, 2024 •
edited by github-actions bot
Loading

Ticket

See also: #141

Resolves navapbc/template-infra#690

Context

See #141 for a comprehensive rundown of the changes. This PR includes a hardcoded exception to make this PR branch mimic a main branch. That is, to make it deploy the Email Notifications resources.

Testing

image

Preview environment

rocketnova and others added 30 commits August 14, 2024 12:15
@rocketnova
Bring over WIP from template-infra branch
ab8e5a4
@rocketnova
Enable notifications
9ab6473
@rocketnova
Restrict access
e772c1e
@rocketnova
Conditionally return values based on verification method
18ee918
@rocketnova
Deal with pinpoint_email_channel role_arn
d62b062
@rocketnova
Separate a verified sender identity from a pinpoint app
391b603
@rocketnova
Refactor outputs and variables
9a607bb
@rocketnova
Add code for aws_sesv2_email_identity_policy
0c417ad
@rocketnova
Drop workspace prefix for notifications config
179b385
@rocketnova
Connect notifications and email-identity modules with app-config and …
79f1bb9 
…app/service
@rocketnova
Rework idp to remove dependency on data output which would otherwise …
c07cb33 
…require first running a -target run
@rocketnova
Add test email to satisfy tests
11eee11
@rocketnova
Fix IAM role name collision
a3e203b
@rocketnova
Grant Github Actions auth for notifications: ses & mobiletargeting
c649240
@rocketnova
Remove deprecated pinpoint identity role
5dbcda6
@rocketnova
Move email identity into resources subdir
f0a9759
@rocketnova
Refactor to support re-using existing email identities
7330a31
@rocketnova
Fix depends_on syntax
d7b5e35
@rocketnova
Simplify
3a80dbf
@rocketnova
Simplify and use domain name
f6a57ed
@rocketnova
Auto-add dns records for domain verification
06bae3a
@coilysiren
Merge branch 'main' into rocket/notifications
9a53c2c
@coilysiren
diff reduction
1e24692
@coilysiren
spacing and docs
64268b1
@coilysiren
use jsonencode
46dd4cd
@coilysiren
module path
0e4eb4c
@coilysiren
prefix
a3834ee
@coilysiren
try to fix verification records
2b7bda9
@coilysiren
try using count
d2c2bf3
@coilysiren
use route53 config from truss
33e654d
@coilysiren coilysiren mentioned this pull request Nov 22, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lorenyu lorenyu lorenyu left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Email notifications
3 participants
@coilysiren @lorenyu @rocketnova